Skip to content

Separate development guide from README#20

Merged
strzibny merged 3 commits intoserpapi:masterfrom
strzibny:development-guide
Mar 16, 2026
Merged

Separate development guide from README#20
strzibny merged 3 commits intoserpapi:masterfrom
strzibny:development-guide

Conversation

@strzibny
Copy link
Contributor

Let's start cleaning up the README by separating the developer guide. I moved it to a separate file and I also removed the UML diagram entirely as it suggests that the workflow is complicated where in fact the gem is very simple which is actually a positive thing.

@strzibny strzibny force-pushed the development-guide branch from 033beac to 9d986a9 Compare March 12, 2026 11:03
@strzibny strzibny requested a review from jvmvik March 12, 2026 11:04
Copy link
Contributor

@trusche trusche left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good idea and best practice, thanks @strzibny. Just some small suggestions

@strzibny strzibny force-pushed the development-guide branch from 5cf5cbe to 1194b14 Compare March 16, 2026 10:58
@strzibny
Copy link
Contributor Author

@trusche Done

Copy link
Contributor

@trusche trusche left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀 Thanks @strzibny

@strzibny strzibny merged commit 0ea7f42 into serpapi:master Mar 16, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants